MCH: improve raw decoder diagnostics - #9881
Conversation
57a968f to
becc679
Compare
|
Error while checking build/O2/o2-cs8 for becc679 at 2022-09-22 17:43: Full log here. |
|
Error while checking build/O2/o2-dataflow-cs8 for becc679 at 2022-09-21 11:51: Full log here. |
|
Error while checking build/AliceO2/O2/o2/macOS for becc679 at 2022-09-21 12:14: Full log here. |
|
Error while checking build/AliceO2/O2/o2/macOS-arm for becc679 at 2022-09-21 12:30: Full log here. |
|
Error while checking build/O2/o2 for becc679 at 2022-09-22 08:23: Full log here. |
|
Error while checking build/O2/o2-dataflow for becc679 at 2022-09-21 14:06: Full log here. |
|
Error while checking build/O2/fullCI for becc679 at 2022-09-22 09:36: Full log here. |
|
Error while checking build/O2/o2-cs8 for 59697c0 at 2022-10-01 09:06: Full log here. |
|
Error while checking build/AliceO2/O2/o2/macOS-arm for 59697c0 at 2022-09-29 20:58: Full log here. |
|
Error while checking build/O2/o2-dataflow-cs8 for 59697c0 at 2022-10-01 04:23: Full log here. |
|
@aferrero2707 I assume this one is a) ready to be merged (no more updates to come ?) b) can be squashed ? Let me know. |
|
@aphecetche yes, ready to be squashed and merged! The CI failures seem to have nothing to do with the changes in this PR... |
- badly formatted SYNC words
- SAMPA packet headers consistency
- chip address and e-link index compatibility
- num of 10-bit words multiple of 4 in cluster sum mode
This is because the corresponding "Bad digit time" error type has been removed, because it is redundant. The digit time can be checked offline with `digit.getTime() != DataDecoder::tfTimeInvalid`
Head branch was pushed to by a user without write access
59697c0 to
22d4fdb
Compare
|
@aphecetche I found a small bug in the HB packets handler, see last commit. If it's ok for you could you re-approve the PR? Thanks! |
The following decoding errors can now be detected and reported:
The check of the digits time in DataDecoder has also been removed, because the corresponding "Bad digit time" error type has been removed as well. The digit time can be checked offline with
digit.getTime() != DataDecoder::tfTimeInvalid.